Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add galoshes to janidrobe +galoshes 1984 #1269

Merged
merged 7 commits into from
Aug 15, 2023

Conversation

mhamsterr
Copy link
Contributor

@mhamsterr mhamsterr commented Aug 5, 2023

На самом деле это revert 13602
Based on https://discord.com/channels/919301044784226385/1137384282122362910

Описание PR

Причина проста: Нанятые главой персонала уборщики могли получить свои галоши (как-бы самый важный элемент их униформы, который в несколько раз упрощал их работу) только через: щитспавн, аплинк (при условии что они являлись синдикатовцами) или в качестве подарка от раундстарт уборщиков. Этот пр должен исправить это недоразумение.

13602 был принят только потому, что на Wizden серверах происходили раундстартовые взломы каморки уборщиков ради этого аналога ноуслипов. У нас же такие действия караются (должны) модерацией как минимум по правилам 3 и 3.1, так что такой проблемы не возникает (возникать не должно).
Проверки

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

🆑 mhamster

  • tweak: Галоши уборщика снова доступны в УборШкафе.
  • tweak: Галоши уборщика получили небольшой штраф к скорости. For the MRP!

Replace ClothingShoesColorBlack with ClothingShoesGaloshes
Based on https://discord.com/channels/919301044784226385/1137384282122362910
@mhamsterr mhamsterr requested a review from Morb0 as a code owner August 5, 2023 15:47
@github-actions github-actions bot added Status: Needs Review This PR requires new reviews before it can be merged No C# Code has not been changed labels Aug 5, 2023
@mhamsterr mhamsterr changed the title Replace black shoes in janidrobe with galoshes Add galoshes to janidrobe Aug 5, 2023
@Myakot
Copy link
Contributor

Myakot commented Aug 5, 2023

Я правильно понимаю - добавим больше работы модерации, но зато упростим жизнь лейтджойнерным уборщикам? :)

@mhamsterr
Copy link
Contributor Author

Я правильно понимаю - добавим больше работы модерации, но зато упростим жизнь лейтджойнерным уборщикам? :)

Не совсем понимаю, как простой вопрос в ахелп "Зачем вы взламываете чулан уборщиков?" добавляет работы модерации? Это скорее выглядит как ежедневная рутина, хз.
К тому же, кто в здравом уме будет вместо работы и выполнения своих обязанностей заниматься взломом, на секундочку, шлюза на космической станции ради пары резиновых сапог? По моему скромному мнению (я никогда не был модератором кстати), тут ну вот как-бы явное нарушение какой-либо логики персонажа...

@Morb0
Copy link
Member

Morb0 commented Aug 5, 2023

Просто добавь им замедления и они никому не будут нужны кроме уборщиков.

@mhamsterr mhamsterr changed the title Add galoshes to janidrobe Add galoshes to janidrobe +galoshes 1984 Aug 6, 2023
@lzk228
Copy link
Collaborator

lzk228 commented Aug 7, 2023

сначала попробуй закинуть визардам замедление галошей

@Morb0
Copy link
Member

Morb0 commented Aug 9, 2023

сначала попробуй закинуть визардам замедление галошей

space-wizards/space-station-14#7793

@lzk228
Copy link
Collaborator

lzk228 commented Aug 9, 2023

сначала попробуй закинуть визардам замедление галошей

space-wizards/space-station-14#7793

ну, уже больше года прошло, может выйдет сейчас это протолкнуть

Co-authored-by: Morb <[email protected]>
@Morb0
Copy link
Member

Morb0 commented Aug 9, 2023

сначала попробуй закинуть визардам замедление галошей

space-wizards/space-station-14#7793

ну, уже больше года прошло, может выйдет сейчас это протолкнуть

Можешь попробовать

@mhamsterr
Copy link
Contributor Author

сначала попробуй закинуть визардам замедление галошей

space-wizards/space-station-14#7793

ну, уже больше года прошло, может выйдет сейчас это протолкнуть

Можешь попробовать

Что-то мне подсказывает, что они не оценят...
И что мне тогда с этим пр-ом делать? Закрывать, или пусть дальше лежит?

@LightVillet
Copy link
Contributor

сначала попробуй закинуть визардам замедление галошей

space-wizards/space-station-14#7793

ну, уже больше года прошло, может выйдет сейчас это протолкнуть

Можешь попробовать

Что-то мне подсказывает, что они не оценят...
И что мне тогда с этим пр-ом делать? Закрывать, или пусть дальше лежит?

А может залить всё-таки пр оффам, при этом указать зачем сделано замедление (для того, чтобы галоши в убормате были)? Или одним пром это сделать

@Morb0 Morb0 merged commit f0546b8 into space-syndicate:master Aug 15, 2023
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No C# Code has not been changed Status: Needs Review This PR requires new reviews before it can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants